Skip to content

[dgb] #82 won-block dispatch-wiring -> master (re-PR of #167) - #198

Merged
frstrtr merged 1 commit into
masterfrom
dgb/won-block-dispatch-wiring
Jun 19, 2026
Merged

[dgb] #82 won-block dispatch-wiring -> master (re-PR of #167)#198
frstrtr merged 1 commit into
masterfrom
dgb/won-block-dispatch-wiring

Conversation

@frstrtr

@frstrtr frstrtr commented Jun 19, 2026

Copy link
Copy Markdown
Owner

Reconciliation re-PR per integrator. dgb/won-block-dispatch-wiring -> master (original #167 was merged into intermediate branch dgb/won-block-broadcaster, never to master).

Stack order: #163 (coin-node-submit-seam) -> #166 (block_broadcast) -> THIS (won_block_dispatch). All three are genuinely MISSING from master (git cherry: 3x +). Lower commits appear in this diff until #163/#166 land, then collapse.

No shared-base/CMake-allowlist concerns beyond what #166/#163 already carry. GPG-signed commits preserved.

make_on_block_found turns a winning share hash into a broadcast: reconstruct
share->block via an injected WonBlockReconstructor, then fire broadcast_won_block
down both paths (P2P primary + external digibyted submitblock fallback). The
reconstructor is injected so the run-loop dispatch contract is build-verified and
run-tested now, before the faithful as_block reassembly is ported.

won_block_dispatch_test drives the handler end-to-end: known share fires both
paths, unknown/unassemblable share broadcasts nothing, RPC-only still reaches the
network, missing reconstructor is a safe no-op. 4/4 PASS. Registered in
test/CMakeLists.txt and both build.yml --target allowlists (avoids the #143
NOT_BUILT trap). src/impl/dgb/ only; no p2pool-merged-v36 surface.
@frstrtr
frstrtr force-pushed the dgb/won-block-dispatch-wiring branch from 5845422 to 67512ef Compare June 19, 2026 09:04
@frstrtr
frstrtr merged commit c1247e2 into master Jun 19, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant